Skip to content

Conversation

arsenm
Copy link
Contributor

@arsenm arsenm commented Sep 8, 2025

There is only one pointer size so there is no reason to use ptr_rc.

There is only one pointer size so there is no reason to use ptr_rc.
Copy link
Contributor Author

arsenm commented Sep 8, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@arsenm arsenm added the backend:VE label Sep 8, 2025 — with Graphite App
@arsenm arsenm requested a review from kaz7 September 8, 2025 07:14
@arsenm arsenm marked this pull request as ready for review September 8, 2025 07:14
Copy link
Contributor

@s-barannikov s-barannikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove VERegisterInfo::getPointerRegClass as well?

@arsenm
Copy link
Contributor Author

arsenm commented Sep 8, 2025

Remove VERegisterInfo::getPointerRegClass as well?

It's not quite dead yet. It's still used when parsing inline assembly memory constraints

@arsenm arsenm merged commit 61a8d2d into main Sep 8, 2025
13 checks passed
@arsenm arsenm deleted the users/arsenm/ve/stop-using-ptr-rc branch September 8, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants